Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@eslint.config.mjs`:
- Around line 6-11: 現在の自動有効化ロジック (laststanceReactNextRules =
Object.fromEntries(Object.keys(laststanceReactNextPlugin.rules ?? {}).map(...)))
はプラグイン更新で新ルールが勝手に error 有効化されCI破壊を招くため、laststanceReactNextPlugin.rules
の全キーを使う代わりに明示的なルールリストを用いるよう修正してください — 具体的には laststanceReactNextRules
を自動生成から、既知の許可されたルール名の配列（whitelistedRules など）を定義してその配列を map/fromEntries して
'error' を割り当てるか、既存の Object.keys(...) をフィルタして許可リストに存在するものだけを有効化する実装に置き換えてください。

In `@src/renderer/settings/sections/General.tsx`:
- Around line 161-172: handleSaveCurrentSize currently awaits
window.electron.window.getMainBounds() without catching a rejection, causing
unhandled promise rejections when invoked via handleSaveCurrentSizeClick; wrap
the await in a try/catch inside handleSaveCurrentSize, and in the catch
setIsMainWindowAvailable(false) and return (optionally log the error), ensuring
updateSettings({ windowSize: bounds }) only runs on success and existing
handleSaveCurrentSizeClick can continue to call void handleSaveCurrentSize().

In `@src/renderer/src/components/sidebar/SourceCard.tsx`:
- Around line 51-57: handleRefresh currently fires multiple thunks via
Promise.all but doesn't detect rejection; change handleRefresh into an async
callback that awaits Promise.all of the dispatched thunks' unwrap() results
(e.g., await Promise.all([dispatch(fetchSourceStats()).unwrap(),
dispatch(fetchSkills()).unwrap(), dispatch(fetchAgents()).unwrap()])) and wrap
that await in try/catch to handle failures (notify user/log error inside catch).
Keep useCallback and dispatch references (handleRefresh, fetchSourceStats,
fetchSkills, fetchAgents) and ensure the caller still uses void handleRefresh()
if previously used.

In `@src/renderer/src/components/skills/UndoToast.tsx`:
- Around line 32-34: The JSDoc for UndoToast is out of date: update comment text
around the toastId usage to remove references to onUndoComplete and instead
document that UndoToast (used by MainContent) will dismiss via
toast.dismiss(toastId); specifically, edit the JSDoc entries that mention
onUndoComplete (the blocks around lines referencing the example/description) to
explain the explicit Sonner id (toastId) and show using toast.dismiss(toastId)
as the undo completion mechanism so the docs match the UndoToast component and
its toastId prop.

In `@src/renderer/src/components/UpdateToast.tsx`:
- Around line 41-43: The close-button handler handleCloseButtonClick is not
memoized like the other handlers; either wrap it with useCallback (e.g., create
handleCloseButtonClick = useCallback(() => handleDismiss(), [handleDismiss])) to
match the pattern used for other handlers, or remove the wrapper entirely and
pass handleDismiss directly to the onClick prop; update the reference in the
component (where handleCloseButtonClick is used) to reflect the chosen approach
so handlers remain consistent and stable.

---

Outside diff comments:
In `@src/renderer/src/components/layout/MainContent.tsx`:
- Around line 404-423: The current handleToastDismissed unconditionally
dispatches clearUndoToast(), which can clear a newer toast's state; update
handleToastDismissed to dispatch clearUndoToast with the toastId (use the
toastId constant captured by the closure) and change the ui reducer
clearUndoToast to accept a payload { id: ToastId } and only clear
state.undoToast when state.undoToast?.id === action.payload.id; keep the toast
invocation (toast(..., { id: toastId, duration: UNDO_WINDOW_MS, onDismiss:
handleToastDismissed, onAutoClose: handleToastDismissed })) and ensure the
reducer name clearUndoToast and the toastId variable are used to locate the
changes.

In `@src/renderer/src/components/skills/UnlinkDialog.tsx`:
- Around line 107-111:
getUnlinkCopyで毎レンダー新しいオブジェクトを作っているためhandleUnlinkの依存配列に入れるとコールバックが毎回再生成されます。解決するには現在のconst
copy =
getUnlinkCopy(...)をuseMemoでラップして、依存関係にvariant、skillToUnlink?.skill.name、skillToUnlink?.symlink.agentNameだけを入れてメモ化し、handleUnlinkの依存配列にはそのメモ化されたcopyを使うようにしてください（参照シンボル:
getUnlinkCopy, copy, handleUnlink）。

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/renderer/src/components/skills/CopyToAgentsModal.tsx`:
- Around line 43-45: selectedAgents is currently local state reset in
CopyToAgentsModal via useComponentEffect (setSelectedAgents) and must be moved
to the skills Redux slice: add selectedAgents state to skillsSlice, implement
actions like toggleSelectedAgent(agentId) and clearSelectedAgents(), update
selectors (selectSelectedAgents) and replace all local
useState/setSelectedAgents usage in CopyToAgentsModal to
useSelector(selectSelectedAgents) and dispatch(toggleSelectedAgent(...)) /
dispatch(clearSelectedAgents()) (including the effect that currently resets on
skillToCopy — replace setSelectedAgents([]) with dispatch(clearSelectedAgents())
inside the useComponentEffect). Ensure existing component logic references the
new selector/action names and remove the local selectedAgents state and its
setters.

---

Outside diff comments:
In `@src/renderer/src/components/skills/UndoToast.tsx`:
- Around line 103-114: The useCallback for handleUndoClick currently mixes a
derived value canUndo with its bases causing needless re-renders; move the
canUndo calculation (isUndoableOperation && !isRestoring && remainingMs > 0)
into handleUndoClick so the callback uses only base values, keep the external
canUndo (used for disabled={!canUndo}) as-is, and update the dependency array of
handleUndoClick to the base symbols [isUndoableOperation, isRestoring,
remainingMs, tombstoneIds, onUndo, toastId] so the callback only re-creates when
underlying inputs change.

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/renderer/src/components/skills/CopyToAgentsModal.tsx`:
- Around line 194-202: Wrap handleRowClick in useCallback to keep consistency
with handleToggle: change the plain function handleRowClick to a memoized
callback (useCallback) that calls handleToggle, with the same dependency set
(agentId, disabled, onToggle) or simply [handleToggle] if you prefer; update the
reference where the intrinsic <div> row click handler uses handleRowClick so it
receives the memoized function. Ensure the exported/local symbol names remain
handleRowClick and handleToggle.
- Line 223: The Checkbox handler handleToggle used in CopyToAgentsModal should
accept the Radix signature (checked: boolean | 'indeterminate') => void instead
of () => void; update the handleToggle function declaration (and any references)
to receive a single parameter typed as boolean | 'indeterminate' and simply
ignore or use it as needed, ensuring the onCheckedChange={handleToggle} is
type-safe in the CopyToAgentsModal component.

---

Outside diff comments:
In `@src/renderer/src/redux/slices/skillsSlice.ts`:
- Around line 449-455: The copyToAgents.fulfilled reducer is clearing
Add-related state (skillToAddSymlinks, selectedAddAgentIds) even though Add and
Copy are independent; update the copyToAgents.fulfilled handler to only clear
copy-related fields (set copying = false, skillToCopy = null,
selectedCopyAgentIds = []), leaving skillToAddSymlinks and selectedAddAgentIds
untouched (match the asymmetry used in createSymlinks.fulfilled which only
clears Add-related state).

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/renderer/src/components/skills/CopyToAgentsModal.tsx`:
- Around line 121-141: The JSX currently computes selection/disabled state
inline inside targetAgents.map using occupiedAgentReasonById, selectedAgents,
copying and isSourceUnavailable and passes them to CopyToAgentOption; extract
that conditional logic into a pure helper in agentSelectionHelpers (e.g., a
function buildAgentViewModel(agent, { occupiedAgentReasonById, selectedAgents,
copying, isSourceUnavailable }) that returns { agentId, name, exists, checked,
disabled, occupiedReason }) and change the map to consume the view model and
only render CopyToAgentOption and pass viewModel fields plus onToggle
(handleAgentToggle); apply the same extraction for the similar block around
lines 236-245 so rendering code is purely presentational and the logic is
unit-testable.

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/renderer/src/components/skills/CopyToAgentsModal.tsx`:
- Around line 46-49: The effect using useComponentEffect currently depends on
the entire skillToCopy object which causes clearCopyAgentSelection() to run on
any reference change; update the dependency to a stable identifier or modal-open
flag instead (e.g., use skillToCopy?.id or an isModalOpen prop/state) so that
clearCopyAgentSelection() only runs when the modal actually opens/closes; keep
the effect and dispatch(clearCopyAgentSelection()) but replace the dependency
array entry skillToCopy with the chosen stable symbol and ensure the modal
open/close lifecycle drives the reset.